GtkFrame: Use style classes for shadow

Use the FLAT style class when shadow is set to none, and
let the theme render the appropriate border.
https://bugzilla.gnome.org/show_bug.cgi?id=732256

GtkEntry: Make has-frame use style classes

Instead of code that internally does weird things, use the FLAT style
class if has-frame is FALSE and remove it otherwise.
Based on a patch by Benjamin Otte.
https://bugzilla.gnome.org/show_bug.cgi?id=732256

fixup entry

GtkButton: Use FLAT style class

This commit makes button always draw background and frame.
Buttons with relief none get a new style class, FLAT, which
allows themes to style these buttons as they like.

We also (finally) mark GTK_RELIEF_HALF as deprecated. It
has never done anything different from GTK_RELIEF_NORMAL.

https://bugzilla.gnome.org/show_bug.cgi?id=732256

Add a FLAT style class

This will be used for frame-less buttons and frames in the future.

GtkScrolledWindow: Use style classes for shadow

Translate shadow != None into the FRAME style class.
This doesn't change the style classes used for drawing,
it only sets the style class permanently instead of
saving and restoring in draw().
https://bugzilla.gnome.org/show_bug.cgi?id=732256

GtkViewport: Use style classes for shadow

Translate shadow != None into the FRAME style class.
This doesn't change the style classes used for drawing,
it only sets the style class permanently instead of
saving and restoring in draw().
https://bugzilla.gnome.org/show_bug.cgi?id=732256

gtk: Don't use GObjectClass.constructor

Use the newer constructed instead, which has a fast path in GObject.

Flip GTK_STOCK_CLEAR icon in rtl locales

Adwaita now includes an edit-clear-rtl variant, so lets use it.

gtkwindow: Remove a dumb invalidate_rect from our configure_event

We shouldn't invalidate the entire window when we receive a
ConfigureNotify. That's just broken.

gtkwindow: Remove a vestigal line of painting for OR windows

Since we have a paint clock, we shouldn't be sending out EXPOSE events
for OR windows inside the ALLOCATE cycle. The idea here was that we
would have to wait for a map to get an OR window to paint to, but since
then this has been abstracted away inside GDK and the paint clock.

gtkwindow: Remove an unnecessary set of grip setting

We already update the grip position in _gtk_window_set_allocation, which
is done through the size_allocate above. Receiving a ConfigureNotify
also won't ever change a grip's visibility, so there's no point in
refreshing it.

treeview: Remove configure-event handler as well

colorplane: Don't connect to the configure-event signal

We want to stop people from using configure events
entirely. GtkDrawingArea synthesizes a configure event, but we can just
make the only user of this functionality stop it, and correct the
documentation to not mention it.

docs: Show examples of how to use templates API

The template documentation is lacking inlined examples on how to use the
templates API, like binding children and callbacks. This makes looking
for best practices a bit harder than it ought to be, for a feature this
useful.

gtkwindow: Force CSD on Wayland

We rely on CSD here on the Wayland platform and giving the opportunity
to turn it off doesn't make too much sense.

icon-browser: Add iconbrowser.gresource.xml to EXTRA_DIST

Disable deprecation warnings in tests

We want to test deprecated properties, so failing the tests with
warnings is not productive.

Updated POTFILES.skip

icon-theme: Add another lookup order test

This tests rtl lookup order for non-symbolic icons.

icon-browser: drop icons that were removed from Adwaita

These non-standard icons were dropped from Adwaita. Eventually,
the icon-browser code should be changed to show all standard
icons + whatever is actually in the icon theme, instead of
hardcoding a list of non-standard names.

gesture: Consume the event triggering ::begin if reset within the handler

If the event triggers GtkGesture::begin, and the handler ends up resetting
the gesture (say, due to taking a grab somewhere else within the handler),
still take the event as "managed", as it actually triggered recognition,
even if just to end abruptly.

https://bugzilla.gnome.org/show_bug.cgi?id=731711

icon-browser: fix a typo

icon browser: drop an misnamed icon

small-progress was just my local test icon for scaling of spinners.

Add an icon browser

This is meant to help app authors choose the right icons.

Makefile: Actually dist the high-contrast and dark test files

Adwaita: add inconsistent hover check/radios

Another forgotten file

Forgotten file

GtkButton: Queue a redraw on enter/leave

We currently getting a redraw for buttons, but that is just luck.
Better to make it explicit.

Drop standalone pixbuf-demo

The same demo is included in gtk3-demo. One copy is enough.

GtkToggleButton: Queue a redraw on enter/leave

Without this, we don't redraw consistently for prelight state changes.

Bump version

3.13.3

Avoid deprecation warnings

We know regions are deprecated, no need to warn about it here.

Adwaita: set the correct indicator color for reorderable hover tabs

Adwaita: only side borders for reorderable tabs

Require GLib 2.41.1 (for G_PARAM_EXPLICIT_NOTIFY)

Adwaita: added active state to the switch slider

Adwaita: comment out button separators removal

...after reading #gnome-design scrollbark, since a consensus on how to achieve
this has not been reached yet.

Adwaita: fix view check/radio selected state.

Adwaita: disable separators in buttons

gdkwindow: Don't leak the surface in gdk_cairo_create()

Add a test for async icon loading

gtkmain: Rearrange code

Prevent a segfault in unknown circumstances, and break the ignore
deprecations wrapper out from the if statement so that we don't
cover too much ground by accident.

gtkwidget: Actually mark set_double_buffered as deprecated

Revert "gtkwidget: Widgets are now always double-buffered"

This reverts commit b875572f2af532b0e8f2dc368f8dc824b5f92724.

Apps like Abiword, gnumeric and gnome-chess, and toolkits like
ClutterGTK were all using this for various purposes, and this made them
break. Bring back this feature for now.

It still won't work under Wayland.

gdkwindow: Allow calling gdk_cairo_create outside of a "paint"

Apps had quite a bit of difficulty getting used to the new rules.
While we weren't expecting anything too deadly, it seemed that
gnumeric and Abiword both used gdk_cairo_create like this.

gdkwindow: Remove extra arguments from process_updates_recurse_helper

These can easily be calculated inside the function.

gdkwindow: Don't do some extra work

We can do an early return in some specific cases before we need to
create and clip a cairo region.

gdkwindow: Remove an unused variable

gdkwindow: Make the precondition of gdk_window_add_damage clearer

gdkwindow: Simplify check for damaging offscreen windows

If a window both has an impl and its impl_window is of type offscreen,
that must mean that it is the offscreen window, and the impl window is
itself. We can reduce the indirection here and make it more clear.

gdkwindow: Make gdk_window_add_damage more private

GtkIconTheme: Copy all fields in icon_info_dup

This functionality is only exercised by gnome-shell, currently.
Therefore, forgetting to copy a field here means an instant
gnome-shell crash :-(. More tests needed.

GtkIconTheme: Be more careful when warning

It sucks when printing a warning causes gnome-shell to crash,
so be more careful about icon names being NULL here.

GtkIconTheme: keep misplaced symbolics working

Recent changes made it a breaking mistake to install symbolic icons
of the wrong size into a theme directory, or into the legacy unthemed
icon location.

Since this change affects many apps, do the extra work to keep these
icons working, but emit warnings, in the hope that this will lead to
cleaning up the mess over time.

Move some comments around

Some documentation rewording

Adwaita: Use rtl variants of pan-end when needed

The -gtk-icontheme() function not have any automatism for
symbolic or rtl variants built-in, instead, we expect the theme
to use selectors to provide the right icon for each situation.

widget-factory: Fix another icon size problem

0 is still not a valid icon size.

win32: Drop remnants of def files

We no longer have .def files, so don't refer to them from the
win32 build machinery. Patch by Cilyan Olowen

placessidebar: Use symbolic name for symbolic icon

https://bugzilla.gnome.org/show_bug.cgi?id=731908

Docs: Reword deprecation notes

Mainly, make sure that deprecated properties are
documented as such. In this case: double-buffered.

notify test: Skip GtkWidget::double-buffered

This property is now deprecated, and ignored, thus: no notification.

widget-factory: Fix an icon size problem

0 is not a valid icon size, and setting it has unpredictable
results. Replace it by 1, which is the right icon size to use
in buttons.

GtkCellRendererPixbuf: Set force_scale_pixbuf

The recent icon theme scaling changes make the code more
sensitive to mis-sized icons (e.g. application icons in
the app chooser). A single row whose size gets blown out
of proportion by a big icon is never wanted in a list.
We can avoid this situation by telling GtkIconHelper to
force-scale the pixbuf to the requested size.

GtkIconHelper: use FORCE_SCALE when force_scale_pixbuf is set

When force_scale_pixbuf is set, the icon helper will scale the
icon to the requested size (either the pixel size, or the resolved
icon size), so we can just as well instruct the icon theme code
to do the scaling for us.

GtkIconTheme: Drop an unused variable

We no longer need a separate field for symbolic icon size,
now that we are using the directory size.

GtkIconTheme: Load symbolic icons at the same size

Reuse the scale information that we have from loading icons
normally, when loading a symbolic icon, so that we apply the
same size constraints.

This commit assumes that svgs have the nominal size of the
directory they are in, which will be true for all current
symbolic icons.

Adapt icontheme tests

With our new, improved icon theme spec compliance, some of the
tests need updates to their expected outcomes.

GtkIconInfo: Small struct packing improvement

This makes the GtkIconInfo struct a bit smaller.

GtkIconInfo: Drop an unused field

The GtkIconInfo threshold field was never used, so lets not
carry it around.

GtkIconTheme: Respect limits in theme dirs

Previously, we were taking thresholds and min/max sizes into
account when choosing the best theme dir, but when it came
to loading the icon, we always scaled icons from scalable
directories all the way, ignoring the min/max size limits.

This commit changes things around so that we now load icons
in Threshold directories at their nominal size, and scale
icons in Scalable directories only up to the specified limits.

To override this and keep the previous behaviour of scaling
all the way to the desired size, use the GTK_ICON_THEME_FORCE_SIZE
flag.

gdkwindow: Provide a default implementation of process_updates_recurse

As a quick code cleanup.

gdkwindow-broadway: Move the dirty flag setting to end_paint

This is where it belongs -- process_updates_recurse is the wrong place
for it.

gdkwindow: Move a variable closer to where it's used

To make the code clearer.

gdkwindow: Remove an unused variable

GtkSwitch: Implement hover for GtkSwitch

This makes switches more similar to other activatable widgets
like buttons or scales.

gdkwidow: Make queue_antiexpose optional

gdkwindow-win32: Remove queue_antiexpose implementation

Since the Win32 code never actually called InvalidateRgn or used the
Win32 update area at all, that meant the only thing that could possibly
invalidate the window was the Win32 window manager as part of scrolling
or resizing, which would also send it a WM_PAINT message.

But the WM_PAINT handling called BeginPaint / EndPaint, which clears the
update area completely! We also draw out-of-band, not directly when
handling WM_PAINT, so there's no way that the update area inside the
Win32 WM would match our local one.

There is no possible way that this queue_antiexpose implementation could
do anything. Remove it.

gdkevents-win32: Remove dead code

gdkwindow: Don't bother with a return parameter for queue_antiexpose

Standard refcounting works perfectly well. Don't give us the opportunity
for more memory leaks.

gdkwindow-broadway: Squash memory leak

We don't need to hang on to the update_area.

gdkwindow-broadway: Make queue_antiexpose implementation private

gdkwindow-wayland: Attach new buffers and submit damage in end_paint

process_updates_recurse is simply the wrong place for it to be.

gdkwindow-wayland: Rename listener -> frame_listener

Don't pollute the static namespace here.

docs: add missing apostrophe

gdkwindow: Correct a signature

We removed the parameter from callers and from the implementation, but
not from the signature up top. I didn't notice because the branch I was
working on removed the signature entirely.

gdkwindow: Remove an extra region copy

This code is only called with the current paint region as its argument.
Instead of having to copy it and do a no-op intersect against itself,
just use the current paint directly.

gdkwindow: Clean up code

Introduce a temporary to make the code a bit more clear under these cases.

gdkwindow: Clean up formatting

Tabs to spaces, add some whitespace.

gdkwindow: Add some extra documentation to gdk_cairo_create

Mention the need to wrap the function inside
gdk_window_begin_paint_region and gdk_window_end_paint.

widgetpath: Deprecate regions

This is a foolowup to d80bf0790d0903ba2825f306b6f7435529f922e2

gtkwidget: Add a warning about set_double_buffered

As requested on the mailing list.

testwindows: Remove another reference to set_double_buffered

gtktrayicon-x11: Stop using set_double_buffered

There's nothing better we can do for this case, now that we always
redirect drawing to a temporary pixmap. Maybe since this is already
X11-specific code, we should just do everything with Xlib directly.

Adwaita: Fix highlight on check buttons

The .button style class does not apply to check and radio
buttons, and .check only applies to menuitems.

gdkwindow: Squash a leak in the subsurface case

cairo_surface_create_for_rectangle takes a ref on the parent surface,
so we need to drop ours.

Rename get_window_surface to ref_window_surface to make the code more
clear and to stop this error from happening again.

gdkwindow: Don't create a subsurface if we don't need to

In the case we have a native window, we don't need to crop it to a
non-native subrectangle, as abs_x / abs_y / width / height are all 0.

gdkwindow: Remove the ability to call begin_paint_region more than once

Previously, each begin_paint_region added to a stack of current paints,
and when end_paint was called, the paint was popped off of the stack and
the surface was composited into the parent paint surface.

However, the code was broken in the case of a backend like Wayland which
didn't keep track of nested calls and simply wiped and returned the
native impl backing surface every time.

Since this feature is flat out unused by GTK+ and we don't want to
really support tricksy things like these for other clients, just remove
the feature. If somebody does call begin_paint_region more than once,
warn and return without doing anything.